Skip to content

Conversation

JoeyMckenzie
Copy link
Contributor

Great work with these kits! One minor thing, looks like the bootstrap/ssr folder isn't being ignored by source control. Added that folder to the .gitignore file to fix that.

Same change can be found in the Vue starter kit as well.

Copy link
Contributor

@tnylea tnylea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Appreciate it!

@tnylea tnylea added the Approved Approved for Merge label Feb 25, 2025
@rahmat-dev
Copy link

rahmat-dev commented Feb 26, 2025

did you get this error after running the ssr command? @JoeyMckenzie @tnylea
image

@JoeyMckenzie
Copy link
Contributor Author

@rahmat-dev Yeah, I noticed that last night. Working to fix that now, just need to adjust the JS execution context for SSR in some scenarios (like the theme toggle that caused that error). Work in progress.

@taylorotwell taylorotwell merged commit 1fad628 into laravel:main Feb 28, 2025
2 checks passed
JoeyMckenzie added a commit to JoeyMckenzie/laravel-official-react-starter-kit that referenced this pull request Mar 2, 2025
* chore: add ssr folder to gitignore

* Update .gitignore

---------

Co-authored-by: Taylor Otwell <[email protected]>
lucianodigital2b referenced this pull request in lucianodigital2b/clevernote Jul 1, 2025
* chore: add ssr folder to gitignore

* Update .gitignore

---------

Co-authored-by: Taylor Otwell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Approved for Merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants